OCPBUGS-86961: CI job kubevirt-metal-ovn-multinet fails to install MetalLB operator#80167
OCPBUGS-86961: CI job kubevirt-metal-ovn-multinet fails to install MetalLB operator#80167amogh-redhat wants to merge 2 commits into
Conversation
|
Skipping CI for Draft Pull Request. |
|
@amogh-redhat: This pull request references Jira Issue OCPBUGS-86961, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughThis PR adds matching Konflux deployment flags and operator source settings to two Hypershift MCE periodic Prow job environment blocks. ChangesMCE Konflux environment configuration
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested labels: 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
/hold |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-kubevirt-metal-ovn-multinet |
|
@amogh-redhat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
4075b2f to
ca01e6f
Compare
|
/lgtm The workflow already has the required: So, adding this variable should suffice. |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-kubevirt-metal-ovn-multinet |
|
@mgencur: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: amogh-redhat, mgencur The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Actually, the tests require all of these: In the latest run, the CatalogSource was not deployed |
ca01e6f to
a0462ac
Compare
|
New changes are detected. LGTM label has been removed. |
|
Thank you @mgencur . I have added the variables. |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-kubevirt-metal-ovn-multinet |
|
@amogh-redhat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
The error - "Nested cluster's node count is not equal to the desired replicas in the NodePool. Retrying in 30 seconds." doesn't look to be an issue with the changes I made. It looks more of an issue with an infrastructure/resource issue. I will initiate the test again. |
|
/retest |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-kubevirt-metal-ovn-multinet |
|
@amogh-redhat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
You're on a good way. The metallb install step already passed in the latest run. |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-kubevirt-metal-ovn-multinet |
|
@amogh-redhat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-kubevirt-metal-ovn-multinet |
|
@amogh-redhat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/retest |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-kubevirt-metal-ovn-multinet |
|
@amogh-redhat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
@amogh-redhat: This pull request references Jira Issue OCPBUGS-86961, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-kubevirt-metal-ovn-multinet |
|
@amogh-redhat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-kubevirt-metal-ovn-multinet |
|
@amogh-redhat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
@amogh-redhat: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
What this PR does / why we need it :
The following jobs fails to install MetalLB operator:
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-kubevirt-metal-ovn-multinet/2060843873688621056
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-kubevirt-metal-ovn-multinet-default-net/2060934532646834176
The configurations for the jobs are stored in “ci-operator/config/openshift/hypershift/openshift-hypershift-release-4.22__periodics-mce.yaml” .
METALLB_OPERATOR_SUB_SOURCE variable is missing for the below tests:
e2e-kubevirt-metal-ovn-multinet
e2e-kubevirt-metal-ovn-multinet-default-net
This is the reason why the job looks for METALLB_OPERATOR_SUB_SOURCE value in the workflow yaml - “ci-operator/step-registry/hypershift/mce/kubevirt/baremetalds/conformance/hypershift-mce-kubevirt-baremetalds-conformance-workflow.yaml”.
The METALLB_OPERATOR_SUB_SOURCE value is defined as “qe-app-registry” in the workflow yaml.
This is causing the error, as the OLM operator couldn’t find “metallb-operator” in “qe-app-registry”.
To fix this, i have adds the “METALLB_OPERATOR_SUB_SOURCE: metallb-konflux” for both the tests mentioned above under the “env:”.
Other tests that are running for the same version already have this variable under "env".
example - e2e-kubevirt-metal-ovn, e-agent-connected-ovn-ipv4-metal-conformance.
Which issue(s) this PR fixes: Fixes - https://redhat.atlassian.net/browse/OCPBUGS-86961
Checklist:
Subject and description added to both, commit and PR.
Relevant issues have been referenced.
This change includes docs.
This change includes unit tests.
Summary by CodeRabbit
This PR updates OpenShift CI configuration for HyperShift MCE periodic jobs (
ci-operator/config/openshift/hypershift/openshift-hypershift-release-4.22__periodics-mce.yaml) to fix MetalLB operator installation failures in the KubeVirt baremetal multinetwork conformance jobs (e2e-kubevirt-metal-ovn-multinetande2e-kubevirt-metal-ovn-multinet-default-net).What changed (practical impact):
envblocks were expanded to enable Konflux-driven deployment of catalogs/operators and to point operator sources at the expected Konflux catalogs:KONFLUX_DEPLOY_CATALOG_SOURCE: "true"KONFLUX_DEPLOY_OPERATORS: "true"KONFLUX_DEPLOY_SUBSCRIPTION: "false"METALLB_OPERATOR_SUB_SOURCE: metallb-konflux(MetalLB now comes from themetallb-konfluxsource)LOCAL_STORAGE_OPERATOR_SUB_SOURCE: local-storage-konflux(local-storage operator source added alongside MetalLB)Scope/impact: